ht4 done#43
Conversation
romabelka
left a comment
romabelka
left a comment
There was a problem hiding this comment.
почитай, пожалуйста, слак. Выполняй домашку поверх моих изменений, проверить ДЗ из 29 измененных файлов невозможно. В следующий раз я не буду принимать в таком виде
| handleAddComment = () => { | ||
| const { addComment, articleId } = this.props | ||
| addComment({ | ||
| "user": this.refs.commentName.value, |
There was a problem hiding this comment.
Не трогай ref без крайней необходимости. Храни значения в state
| } | ||
|
|
||
| state = { | ||
| textIsEmpty: true, |
There was a problem hiding this comment.
у меня концептуально в форме кнопка "отправить" становится доступной только при заполнении обоих полей.
Для этого тут и работа с рефами, чтобы данные формы подтягивать и стейт текущего состояния инпутов и все такое прочее.
И обнуление заполненности полей через реф тоже за этим.)
Все равно неверно?)
| "text": this.refs.commentText.value | ||
| }, articleId) | ||
|
|
||
| this.refs.commentName.value = '' |
| render() { | ||
| const { articles, selected } = this.props | ||
| const options = articles.map(article => ({ | ||
| const options = Object.values(articles).map(article => ({ |
There was a problem hiding this comment.
Лучше селектор, который достанет из в виде массива
| const {type, payload} = action | ||
|
|
||
| switch (type){ | ||
| case ADD_COMMENT: |
There was a problem hiding this comment.
через мидлвары будет проходить каждый экшин, они должны быть максимально общими, завязывать на конкретные экшины - не лучшее решение
|
|
||
| switch (type){ | ||
| case ADD_COMMENT: | ||
| payload.comment.id = makeid() |
There was a problem hiding this comment.
лучше не мутировать payload, мало-ли что там станут передавать
2 participants
No description provided.